Skip to content

fix(log): normalize client namespace filters#31

Open
taigrr wants to merge 1 commit into
masterfrom
cd/round4-log-socket
Open

fix(log): normalize client namespace filters#31
taigrr wants to merge 1 commit into
masterfrom
cd/round4-log-socket

Conversation

@taigrr

@taigrr taigrr commented Jun 23, 2026

Copy link
Copy Markdown
Owner

Summary

  • trim, drop blank, and deduplicate namespace filters passed directly to log.CreateClient
  • ensure blank-only filters keep the documented all-namespaces behavior
  • make the whitespace-trimming websocket namespace test deterministic under race runs

Verification

  • go generate ./...
  • go test -race ./...
  • staticcheck ./...
  • go test -race ./ws -run TestLogSocketHandler_NamespaceFilter_TrimsWhitespace -count=20

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant